-
Notifications
You must be signed in to change notification settings - Fork 54
Add support for discovery extensions #760
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
83825d3 to
a9277d8
Compare
87a764c to
27e0ae6
Compare
|
Currently, the implementation requires returning the absolute path to a resource manifest - but I Right now, it expects JSON like this: {"resourceManifestPath":"/foo/bar/baz.dsc.resource.json"}But we could have it return this, and parse the last segment of the path to determine kind: {"absoluteManifestPath":"/foo/bar/baz.dsc.extension.json"}Or require it return either If we support extensions discovering extensions, we would need to recursively handle extension |
|
I think we can defer extension discovery til when we need it |
e1cdb55 to
233c3f5
Compare
|
Renamed from |
233c3f5 to
014a16f
Compare
PR Summary
dscto list extensions:dsc extension listdiscovercapability currently supported.The output from a discovery extension must match:
{ "manifestPath": "<absolute path>" }PR Context
Fix #681